Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add rootlets midpoints file #31

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add rootlets midpoints file #31

wants to merge 5 commits into from

Conversation

sandrinebedard
Copy link
Member

@sandrinebedard sandrinebedard commented Nov 12, 2024 •
edited
Loading

Description

The following commands were used to create PAM50_rootlets_midpoints.nii.gz:

# get center-of-mass of rootlets
sct_label_utils -i PAM50_rootlets.nii.gz -cubic-to-point -o  PAM50_rootlets_mid.nii.gz

# project on spinal cord centerline
sct_label_utils -i PAM50_centerline.nii.gz -project-centerline PAM50_rootlets_mid.nii.gz  -o PAM50_rootlets_midpoint.nii.gz

@sandrinebedard sandrinebedard marked this pull request as ready for review December 17, 2024 18:49
jcohenadad
jcohenadad approved these changes Dec 17, 2024
View reviewed changes
Copy link
Member

@jcohenadad jcohenadad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you!

@joshuacwnewton joshuacwnewton mentioned this pull request Dec 18, 2024
Open
8 tasks
joshuacwnewton
joshuacwnewton requested changes Dec 18, 2024
View reviewed changes
Copy link
Member

@joshuacwnewton joshuacwnewton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one small change + a question :)

template/info_label.txt Show resolved Hide resolved
template/info_label.txt Outdated Show resolved Hide resolved
template/info_label.txt Outdated Show resolved Hide resolved
joshuacwnewton
joshuacwnewton approved these changes Dec 18, 2024
View reviewed changes
Copy link
Member

@joshuacwnewton joshuacwnewton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joshuacwnewton joshuacwnewton joshuacwnewton approved these changes

@jcohenadad jcohenadad jcohenadad approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sandrinebedard @jcohenadad @joshuacwnewton